feat: add rainbow wave animation to progress bar#28
Open
AurelianoBR wants to merge 4 commits intoglauberlima:mainfrom
Open
feat: add rainbow wave animation to progress bar#28AurelianoBR wants to merge 4 commits intoglauberlima:mainfrom
AurelianoBR wants to merge 4 commits intoglauberlima:mainfrom
Conversation
Replace static tier-based coloring with a per-character rainbow wave using ANSI 256-color codes. Each filled block gets a different color from a 12-color palette, and the phase shifts once per second based on epoch time, creating a flowing wave effect across invocations. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- SC2250: add braces around $color_idx and $_v variable references - SC2310: extract is_present/validate_directory calls from if conditions to avoid set -e being disabled inside conditionals Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Use `|| rc=$?` pattern to capture return codes outside any conditional chain, satisfying shellcheck SC2310 which warns about set -e being disabled inside if/&& conditions. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Add SHOW_RAINBOW_WAVE toggle to the config patching system, allowing users to opt-out of rainbow wave animation during installation. When disabled, falls back to tier-based color coding for the progress bar. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
SHOW_RAINBOW_WAVEtoggle (enabled by default)install.shfeature selection menu andpatch-statusline.shpatching systemChanges
WAVE_COLORSarray,SHOW_RAINBOW_WAVEconfig flag, and conditional rendering inbuild_progress_bar(). Isolateis_present()calls from conditional contexts to satisfy SC2310.--no-rainbow-waveflag and includeSHOW_RAINBOW_WAVEin config block patching.rainbow-waveto component selection menu (7 options, up from 4). Default selection includes all features.Test plan
./tests/unit.sh && ./tests/integration.sh && ./tests/shellcheck.sh— all passSHOW_RAINBOW_WAVE=true— verify rainbow colors cycle through filled blocksSHOW_RAINBOW_WAVE=false— verify fallback to tier-based solid colors./patch-statusline.sh statusline.sh --no-rainbow-wavepatches correctly./install.sh— verify new menu options work for all 7 selections🤖 Generated with Claude Code